Use generator local targetV if set as local regulation #1295
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vidaldid-rte
force-pushed
the
feature/use_local_target_v
branch
from
1240cd1 to
6bdec1c
Compare
vidaldid-rte
changed the title
vidaldid-rte
force-pushed
the
ci/core-7.1.0-SNAPSHOT
branch
2 times, most recently
from
6250ea7 to
4ae5029
Compare
…RemoteControl=false Signed-off-by: Didier Vidal <[email protected]>
…RemoteControl=false Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
vidaldid-rte
force-pushed
the
feature/use_local_target_v
branch
from
8581251 to
54769aa
Compare
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
| if (disableInconsistentVoltageControls && Math.abs(localTargetV - genLocalTargetV) > TARGET_V_EPSILON) { | ||
| return Double.NaN; | ||
| } | ||
| // else if !disableInconsistentVoltageControls keep the first voltage target found |
Member
There was a problem hiding this comment.
Should we add a warn log/report when detecting inconsistent voltage control (precising if this local voltage control is taken into account or not regarding the parameter disableInconsistentVoltageControl) ?
Collaborator
Author
There was a problem hiding this comment.
Good point. It also joins a remark from Damien that challenged the new equivalentTargetV API in lfGenerator.
I refactored in order to keep only one targetV and reuse all existing check mechanisms.
src/main/java/com/powsybl/openloadflow/network/impl/LfNetworkLoaderImpl.java
Outdated
Show resolved
Hide resolved
| olf.reactiveControllerBusesToPqBuses = ${remoteReactivePowerControllerBusToPqCount} noeuds électriques avec régleur de puissance réactive à distance passé(s) en PQ | ||
| olf.reactiveControllerBusesToPqMaxQ = Réglage de la puissance réactive distante du noeud électrique '${busId}' -> PQ, q=${busQ} > maxQ=${maxQ} | ||
| olf.reactiveControllerBusesToPqMinQ = Noeud électrique ${busId} régleur de puissance réactive à distance -> PQ, q=${busQ} < minQ=${minQ} | ||
| olf.rescaledRemoteVoltageControls = ${count} contrôles de tension distants n'ont pas de consigne équivalente locale et passent en réglage local par mise à l'échelle de la tension nominale locals. |
Member
There was a problem hiding this comment.
"mise à l'échelle de la tension nominale locale."
…oaderImpl.java Co-authored-by: Sylvestre Prabakaran <[email protected]> Signed-off-by: vidaldid-rte <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
SylvestreSakti
previously approved these changes
Dec 1, 2025
jeandemanged
changed the title
8 tasks
) Signed-off-by: Damien Jeandemange <[email protected]>
Signed-off-by: Didier Vidal <[email protected]>
SylvestreSakti
approved these changes
Dec 2, 2025
|
jeandemanged
approved these changes
Dec 2, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
#1268
NOTE:
This PR depends on powsybl/powsybl-core#3635 (review)
What kind of change does this PR introduce?
If a generator is configured for remote voltage control but needs to turn local for any reason and if a local target v is defined, that value is used instead of keeping the same pu target.
Reasons for tunring remote to local include:
parameter voltageRemoteControl set to false
consistency checks in OLF that switch a remote voltage control to a local voltage control
What is the current behavior?
When a generator is moved to local voltage control the remote target in pu is kept locally
What is the new behavior (if this is a feature change)?
If a local target has been defined (by a state esimator or any other process in charge of coputing voltage target) that value is used instead.
Does this PR introduce a breaking change or deprecate an API?